[IMP] Inventory: make Storage Categories current with 18.0 #15090
Conversation
|
|
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Good job testing doing this deep dive, @erjer-odoo !
Appreciate the strenuous testing you did and challenging my misunderstandings of the putaway behavior to result in a more accurate document! To make sure your new findings are properly highlighted, I'd like to suggest we move some things around and map out the workflows in even more detail
Let me know if I can clarify any of my comments!
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
erjer-odoo
force-pushed
the
18.0-Inventory-storage-categories-erjer
branch
from
1d3d322 to
1517ce9
Compare
|
Updated with comments from @Felicious . |
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Great work testing this feature meticulously, @erjer-odoo !
Had super minor things I found and this PR is ready to merge
@robodoo delegate+
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
|
I'd typically rate this as a 2pt update since it mainly brings the doc from version 18 to 19 with under 50% of the lines changed. That said, given the depth of research and testing you invested (especially around a complex feature that even the functional expert found challenging 🫶 ) I’m increasing the value to 3pts. This reflects the extra effort you put into ensuring accuracy and completeness in the original content! |
erjer-odoo
force-pushed
the
18.0-Inventory-storage-categories-erjer
branch
from
1517ce9 to
eec7be4
Compare
|
@robodoo r+ |
closes #15090 Signed-off-by: Erin Jerison (erjer) <erjer@odoo.com>
Making storage categories documentation current with 18.0, requested here:: https://www.odoo.com/odoo/my-tasks/4681301